-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(lit-task-auto-top-up): Update worker to support 1-task-per-network on a single mongodb backend task tracking instance #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nooice! Love the use of recipientDetailSchema
zod schema!
|
||
z.array(recipientDetailSchema).parse(recipientList); | ||
z.array(recipientDetailSchema).parse(deprecatedHabaneroRecipientList); | ||
z.array(recipientDetailSchema).parse(habanberoRecipientList); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
…to register network-specific daily task job name for Agenda - Now, we can run multiple worker instances (1 per network), but persist their jobs to a single backend Mongo instance without issue
…ipient list files in worker dir
…no-specific recipient list files in worker dir
…nts from deprecated generic json file
b64375c
to
58861f6
Compare
Rebased and synced contents of Also bumped contracts-sdk to v6.1.0 to ensure we're using the latest changes re: setting |
With this PR, we can run multiple worker instances (1 per network), but persist their jobs to a single backend Mongo instance without issue.
recipient_list_manzano.json
file in theworker
directory, which our new manzano instance can be configured via env var to load.recipient_list.json
file in theworker
directory, which was habanero specific. The new file includeshabanero
in the same for consistency. Once this is merged and the existing worker is updated to reference the new filename withhabanero
in it, we can delete the existing generically namedrecipient_list.json
file.0x9f32f6555a117ac137775973C6856bC52c249C80
in all 3 files -- this address is a 'stub' address that I use as a canary for the job/worker; it is a wallet that I created. It was set to mint every 5 days. I changed it to mint every 20 instead.